Fix the stay up algorithm for combo boxes to better handle the "click"

Wed Jun  2 01:56:02 2004  Soeren Sandmann  <sandmann@daimi.au.dk>

        * gtk/gtkmenu.c: Fix the stay up algorithm for combo boxes to
        better handle the "click" case. bug  Better fix for #141169.

diff --git a/ChangeLog b/ChangeLog
index da80a898079265102e358ee93c6fa6f429cd9084..6e57016da171f45fbfde4a004ee0983302d4ac90 100644 (file)
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,8 @@
+Wed Jun  2 01:56:02 2004  Soeren Sandmann  <sandmann@daimi.au.dk>
+
+       * gtk/gtkmenu.c: Fix the stay up algorithm for combo boxes to
+       better handle the "click" case. bug  Better fix for #141169.
+
 2004-06-02  Federico Mena Quintero  <federico@ximian.com>
 
        * gtk/gtktreeview.c (gtk_tree_view_search_init): Bail out if the
@@ -59,6 +64,8 @@ Tue Jun  1 22:19:55 2004  Matthias Clasen  <maclas@gmx.de>
 
 Wed Jun  2 00:39:58 2004  Soeren Sandmann  <sandmann@daimi.au.dk>
 
+       Bug #141169.
+
        * gtk/gtkmenushell.c (gtk_menu_shell_button_release): Make menus
        only stay up if you release within 500 milliseconds.
        (gtk_menu_shell_button_press): Set the activate_time to the event

diff --git a/ChangeLog.pre-2-10 b/ChangeLog.pre-2-10
index da80a898079265102e358ee93c6fa6f429cd9084..6e57016da171f45fbfde4a004ee0983302d4ac90 100644 (file)
--- a/ChangeLog.pre-2-10
+++ b/ChangeLog.pre-2-10
@@ -1,3 +1,8 @@
+Wed Jun  2 01:56:02 2004  Soeren Sandmann  <sandmann@daimi.au.dk>
+
+       * gtk/gtkmenu.c: Fix the stay up algorithm for combo boxes to
+       better handle the "click" case. bug  Better fix for #141169.
+
 2004-06-02  Federico Mena Quintero  <federico@ximian.com>
 
        * gtk/gtktreeview.c (gtk_tree_view_search_init): Bail out if the
@@ -59,6 +64,8 @@ Tue Jun  1 22:19:55 2004  Matthias Clasen  <maclas@gmx.de>
 
 Wed Jun  2 00:39:58 2004  Soeren Sandmann  <sandmann@daimi.au.dk>
 
+       Bug #141169.
+
        * gtk/gtkmenushell.c (gtk_menu_shell_button_release): Make menus
        only stay up if you release within 500 milliseconds.
        (gtk_menu_shell_button_press): Set the activate_time to the event

diff --git a/ChangeLog.pre-2-6 b/ChangeLog.pre-2-6
index da80a898079265102e358ee93c6fa6f429cd9084..6e57016da171f45fbfde4a004ee0983302d4ac90 100644 (file)
--- a/ChangeLog.pre-2-6
+++ b/ChangeLog.pre-2-6
@@ -1,3 +1,8 @@
+Wed Jun  2 01:56:02 2004  Soeren Sandmann  <sandmann@daimi.au.dk>
+
+       * gtk/gtkmenu.c: Fix the stay up algorithm for combo boxes to
+       better handle the "click" case. bug  Better fix for #141169.
+
 2004-06-02  Federico Mena Quintero  <federico@ximian.com>
 
        * gtk/gtktreeview.c (gtk_tree_view_search_init): Bail out if the
@@ -59,6 +64,8 @@ Tue Jun  1 22:19:55 2004  Matthias Clasen  <maclas@gmx.de>
 
 Wed Jun  2 00:39:58 2004  Soeren Sandmann  <sandmann@daimi.au.dk>
 
+       Bug #141169.
+
        * gtk/gtkmenushell.c (gtk_menu_shell_button_release): Make menus
        only stay up if you release within 500 milliseconds.
        (gtk_menu_shell_button_press): Set the activate_time to the event

diff --git a/ChangeLog.pre-2-8 b/ChangeLog.pre-2-8
index da80a898079265102e358ee93c6fa6f429cd9084..6e57016da171f45fbfde4a004ee0983302d4ac90 100644 (file)
--- a/ChangeLog.pre-2-8
+++ b/ChangeLog.pre-2-8
@@ -1,3 +1,8 @@
+Wed Jun  2 01:56:02 2004  Soeren Sandmann  <sandmann@daimi.au.dk>
+
+       * gtk/gtkmenu.c: Fix the stay up algorithm for combo boxes to
+       better handle the "click" case. bug  Better fix for #141169.
+
 2004-06-02  Federico Mena Quintero  <federico@ximian.com>
 
        * gtk/gtktreeview.c (gtk_tree_view_search_init): Bail out if the
@@ -59,6 +64,8 @@ Tue Jun  1 22:19:55 2004  Matthias Clasen  <maclas@gmx.de>
 
 Wed Jun  2 00:39:58 2004  Soeren Sandmann  <sandmann@daimi.au.dk>
 
+       Bug #141169.
+
        * gtk/gtkmenushell.c (gtk_menu_shell_button_release): Make menus
        only stay up if you release within 500 milliseconds.
        (gtk_menu_shell_button_press): Set the activate_time to the event

diff --git a/gtk/gtkmenu.c b/gtk/gtkmenu.c
index 0ef126f3adf675d4c5033d55ffd261ba6535a085..3e4c3fa3d5ecc3bd5e52cd0c39decb816fe300b2 100644 (file)
--- a/gtk/gtkmenu.c
+++ b/gtk/gtkmenu.c
@@ -73,11 +73,16 @@ struct _GtkMenuAttachData
   GtkMenuDetachFunc detacher;
 };
 
+typedef enum {
+  NONE,
+  SEEN_MOTION,
+  SEEN_MOTION_ENTER,
+  SEEN_MOTION_ENTER_MOTION
+} SelectState;
+
 struct _GtkMenuPrivate 
 {
-  gboolean seen_motion;
-  gboolean seen_enter;
-    
+  SelectState select_state;
   gboolean have_position;
   gint x;
   gint y;
@@ -1280,8 +1285,7 @@ gtk_menu_popup (GtkMenu               *menu,
   
   menu_shell->parent_menu_shell = parent_menu_shell;
 
-  priv->seen_motion = FALSE;
-  priv->seen_enter = FALSE;
+  priv->select_state = NONE;
   
   /* Find the last viewable ancestor, and make an X grab on it
    */
@@ -2728,11 +2732,18 @@ gtk_menu_motion_notify  (GtkWidget         *widget,
       GtkMenuPrivate *priv = gtk_menu_get_private (GTK_MENU (widget));
       
       gtk_menu_handle_scrolling (GTK_MENU (widget), TRUE);
-      priv->seen_motion = TRUE;
-      if (priv->seen_enter)
+
+      if (priv->select_state == NONE)
        {
-         /* After having seen both a motion event and an enter event,
-          * button releases should be interpreted to mean "activate"
+         priv->select_state = SEEN_MOTION;
+       }
+      else if (priv->select_state == SEEN_MOTION_ENTER)
+       {
+         priv->select_state = SEEN_MOTION_ENTER_MOTION;
+         
+         /* After having seen both a motion, an enter, and a motion
+          * in that order, button releases should be interpreted
+          * as "activate".
           */
          
          GTK_MENU_SHELL (widget)->activate_time = 0;
@@ -2965,6 +2976,7 @@ gtk_menu_enter_notify (GtkWidget        *widget,
 {
   GtkWidget *menu_item;
 
+  menu_item = gtk_get_event_widget ((GdkEvent*) event);
   if (widget && GTK_IS_MENU (widget))
     {
       GtkMenuShell *menu_shell = GTK_MENU_SHELL (widget);
@@ -2973,21 +2985,14 @@ gtk_menu_enter_notify (GtkWidget        *widget,
       if (!menu_shell->ignore_enter)
        gtk_menu_handle_scrolling (GTK_MENU (widget), TRUE);
 
-      priv->seen_enter = TRUE;
-      if (priv->seen_motion)
-       {
-         /* After having seen both a motion event and an enter event,
-          * button releases should be interpreted to mean "activate"
-          */
-         menu_shell->activate_time = 0;
-       }
+      if (priv->select_state == SEEN_MOTION)
+       priv->select_state = SEEN_MOTION_ENTER;
     }
   
   /* If this is a faked enter (see gtk_menu_motion_notify), 'widget'
    * will not correspond to the event widget's parent.  Check to see
    * if we are in the parent's navigation region.
    */
-  menu_item = gtk_get_event_widget ((GdkEvent*) event);
   if (menu_item && GTK_IS_MENU_ITEM (menu_item) && GTK_IS_MENU (menu_item->parent) &&
       gtk_menu_navigating_submenu (GTK_MENU (menu_item->parent), event->x_root, event->y_root))
     return TRUE;